-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
browse
the last deployments
#617
Conversation
Thanks @SteBaum. I just tried and it works. I reconsider what I said during the meeting and would rather have the most recent deployment at the end of the output, it makes more sense in a CLI. @rpignolet was right ! Can you modify it ? Also, I think this is a fix rather than a refactor. |
We should talk about it with @PaulFarault how to fix that (find the good query to do the job) |
8de63fa
to
7d3f55a
Compare
@PACordonnier see if it suits you now. The query returns the last deployments (15 by default) in ascending order. |
Works perfect thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should talk about it with @PaulFarault how to fix that (find the good query to do the job)
This is the correct query to modify. However, the offset
seems to be broken in this new version. It should be located next to the limit
.
Fixes #616 return the last deployments insteads of the firsts
316d5c3
to
9e8d786
Compare
browse
the last deployments
Which issue(s) this PR fixes
Fixes #616
Additional comments
Changed the order in query. Now showing the deployments in descending order.
Agreements